Skip to content

adding dispatch option for rendering build-collection#45

Merged
carriewright11 merged 1 commit into
mainfrom
carriewright11-patch-1
May 12, 2025
Merged

adding dispatch option for rendering build-collection#45
carriewright11 merged 1 commit into
mainfrom
carriewright11-patch-1

Conversation

@carriewright11

@carriewright11 carriewright11 commented May 12, 2025

Copy link
Copy Markdown
Contributor

I noticed that the https://github.com/fhdsl/DaSL_Collection/actions/workflows/build-collection.yml has never run. The plan I think was to be triggered by the other workflows, but I may be misinterpreting.

I am adding a dispatch option in case we want to manually trigger (this is helpful for testing changes in the collection or the code to generate the collection). I wonder if a schedule makes sense for this workflow too (like the render wokflow).

@github-actions

github-actions Bot commented May 12, 2025

Copy link
Copy Markdown
Contributor

No spelling errors! 🎉
Comment updated at 2025-05-12-20:01:34 with changes from b92adf5

@github-actions

github-actions Bot commented May 12, 2025

Copy link
Copy Markdown
Contributor

No broken url errors! 🎉
Comment updated at 2025-05-12-20:01:33 with changes from b92adf5

@carriewright11

Copy link
Copy Markdown
Contributor Author

@avahoffman wanted to make you aware of what I was doing. What do you think about having the build-collection.yml get triggered on a schedule (like the render-site.yml)?

@github-actions

Copy link
Copy Markdown
Contributor

Re-rendered previews from the latest commit:

Updated at 2025-05-12 with changes from b92adf5

@carriewright11 carriewright11 merged commit fa844b0 into main May 12, 2025
12 checks passed
@carriewright11 carriewright11 deleted the carriewright11-patch-1 branch May 12, 2025 20:04
@avahoffman

Copy link
Copy Markdown
Contributor

THanks! totally makes sense.

@carriewright11

Copy link
Copy Markdown
Contributor Author

@avahoffman i see now that it is triggered by the render-site.yml file

@carriewright11

Copy link
Copy Markdown
Contributor Author

so actually maybe we don't need this anymore

@avahoffman

Copy link
Copy Markdown
Contributor

Ah yes, you're right, maybe we revert then?

@carriewright11

Copy link
Copy Markdown
Contributor Author

yup - im on it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants